Skip to content

Conversation

@runcom
Copy link
Member

@runcom runcom commented Sep 13, 2016

some fixes for #297 and another for schema/manifest_test.go which was behaving wrongly. In such cases we expected a given error, test passes but the err we got wasn't the one expected (but yet the test passed)

Note: the img/media-types.png picture has been generated with the Makefile - I don't know why it has a different size

Signed-off-by: Antonio Murdaca <[email protected]>
Signed-off-by: Antonio Murdaca <[email protected]>
@runcom
Copy link
Member Author

runcom commented Sep 13, 2016

@vbatts @philips @stevvooe PTAL

@philips
Copy link
Contributor

philips commented Sep 13, 2016

LGTM

Approved with PullApprove

@wking
Copy link
Contributor

wking commented Sep 13, 2016

3795c5f looks fine to me.

I still think we want to make sure the tooling doesn't choke on
nondistributable 1. However, I'm fine punting that point to after
something like #304 makes the “don't choke” requirement explicit.

@vbatts
Copy link
Member

vbatts commented Sep 13, 2016

LGTM

Approved with PullApprove

@vbatts vbatts merged commit 90334bd into opencontainers:master Sep 13, 2016
@runcom runcom deleted the fix-297 branch September 13, 2016 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants